[Feature] Pipe stacks #1017
[Feature] Pipe stacks #1017
Conversation
Pipe stacks Pipe stacks allow device-on-pipe stacking (Rxup#389) * implement device-on-pipe stack allowage * missing comments
ОбзорWalkthroughИзменения затрагивают систему управления перекрытием труб в игре, вводя новую конфигурационную переменную Changes
Sequence DiagramsequenceDiagram
participant Config as Configuration Manager
participant PipeSystem as PipeRestrictOverlapSystem
participant Pipe as Gas Pipe Entity
Config->>PipeSystem: Update StrictPipeStacking
PipeSystem->>Pipe: Check Overlap Rules
alt Strict Stacking Enabled
PipeSystem-->>Pipe: Prevent Overlapping
else Strict Stacking Disabled
PipeSystem-->>Pipe: Allow Conditional Stacking
end
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
Content.Server/Atmos/EntitySystems/PipeRestrictOverlapSystem.cs (1)
28-28: Рассмотрите использование свойства вместо публичного поля.Публичное поле
StrictPipeStackingможет быть преобразовано в свойствоpublic bool StrictPipeStacking { get; private set; }для лучшей инкапсуляции.Resources/Prototypes/Entities/Structures/Piping/Atmospherics/pipes.yml (1)
68-74: Хорошее архитектурное решение!Создание абстрактной сущности GasPipeNoOverlap с компонентом PipeRestrictOverlap обеспечивает более гибкий контроль над ограничениями наложения труб. Это позволяет выборочно применять ограничения там, где они действительно необходимы.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
Content.Server/Atmos/EntitySystems/PipeRestrictOverlapSystem.cs(4 hunks)Content.Shared/Backmen/CCVar/CCVars.cs(1 hunks)Resources/Prototypes/Entities/Structures/Piping/Atmospherics/binary.yml(1 hunks)Resources/Prototypes/Entities/Structures/Piping/Atmospherics/pipes.yml(7 hunks)Resources/Prototypes/Entities/Structures/Piping/Atmospherics/unary.yml(2 hunks)Resources/Prototypes/Entities/Structures/Power/Generation/teg.yml(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- Resources/Prototypes/Entities/Structures/Piping/Atmospherics/binary.yml
⏰ Context from checks skipped due to timeout of 300000ms (2)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
🔇 Additional comments (15)
Content.Server/Atmos/EntitySystems/PipeRestrictOverlapSystem.cs (9)
7-11: Замечательно!Новые директивы
usingвыглядят корректно и позволяют использовать необходимые конфигурационные классы.
21-21: Полезная зависимость через IConfigurationManager.Поле
_cfgвнедряется корректно, обеспечивая гибкую настройку параметров труб через конфигурацию.
35-35: Убедитесь в корректном управлении событиями при изменении CVar.Проверите, требуется ли отписка от события при завершении работы системы, чтобы избежать потенциальных утечек или лишних колбэков.
86-87: Инициализация переменной выглядит хорошо.Логика флагов направления труб (
takenDirs = PipeDirection.None) понятна и не вызывает вопросов.
97-103: Отличная интеграция вызова PipeNodesOverlap.Использование кортежа
(overlapping, which)упрощает код, делая его более наглядным.
109-110: Расширение функционала через отдельный метод.Возвращаемый кортеж
(bool, PipeDirection)выглядит удобным средством для определения факта пересечения и совокупных направлений.
121-123: Условное поведение при отключённом StrictPipeStacking.Понятно, что если
StrictPipeStacking == false, дальнейшая проверка не требуется. Логика выглядит согласованной.
128-129: Проверьте корректность использования побитового XOR.Проверка
(dir ^ otherDir) != 0приводит к прерыванию цикла при любом отличии битов. Убедитесь, что это ожидаемое поведение и что не нужно учитывать отдельные биты по-другому.
133-134: Итоговое условие выглядит логичным.Если
StrictPipeStackingотключён, проверка(takenDirs & entDirsCollapsed) == entDirsCollapsedкорректно определяет, заняты ли все направления.Content.Shared/Backmen/CCVar/CCVars.cs (1)
206-215: Новая переменная конфигурации предоставляет гибкую настройку поведения труб.
StrictPipeStackingдобавляет возможность жёсткой проверки наложения труб. Значение по умолчаниюfalseуказывает, что изначально жёсткая проверка отключена, что отличный выбор для совместимости.Resources/Prototypes/Entities/Structures/Power/Generation/teg.yml (1)
179-180: Логичное отключение PipeRestrictOverlap для циркулятора.Комментирование компонента
PipeRestrictOverlapсогласуется с новой политикой наложения труб и устройств. Теперь циркулятор можно устанавливать поверх трубы без запрета.Resources/Prototypes/Entities/Structures/Piping/Atmospherics/pipes.yml (2)
79-80: Последовательное применение изменений в иерархии наследования.Все базовые типы труб теперь наследуются от GasPipeNoOverlap, что сохраняет исходное поведение ограничений наложения. Изменения применены последовательно и сопровождаются поясняющими комментариями.
Also applies to: 99-100, 133-134, 174-175, 211-212, 248-249
54-55: Проверьте влияние на архитектуру системы труб.Удаление компонента PipeRestrictOverlap из базового класса GasPipeBase может привести к неожиданному поведению наложения для всех производных сущностей. Рекомендуется проверить, что это изменение не нарушает существующую логику размещения устройств.
✅ Verification successful
Изменение является архитектурно согласованным
Анализ показывает, что отключение компонента PipeRestrictOverlap в базовом классе является частью систематического изменения, которое последовательно применено во всех файлах конфигурации труб. Это подтверждает намеренность архитектурного изменения для поддержки наложения устройств.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Поиск всех сущностей, наследующихся от GasPipeBase ast-grep --pattern 'parent: GasPipeBase'Length of output: 525
Script:
#!/bin/bash # Search for PipeRestrictOverlap in pipe-related files rg "PipeRestrictOverlap" Resources/Prototypes/Entities/Structures/Piping/Length of output: 587
Resources/Prototypes/Entities/Structures/Piping/Atmospherics/unary.yml (2)
218-219: Проверьте игровой баланс.Удаление компонента PipeRestrictOverlap позволит размещать термомашины поверх труб. Убедитесь, что это изменение не создаст нежелательных комбинаций или эксплойтов в игровом процессе.
398-399: Необходимо тестирование размещения конденсаторов.Удаление ограничений на размещение конденсаторов должно быть тщательно протестировано, особенно в сочетании с другими атмосферными устройствами. Рекомендуется проверить:
- Корректность работы конденсатора при размещении на трубах
- Взаимодействие с другими устройствами в той же клетке
|
Вообще, это всё должно лежать в неймспейсе губов, а не Бакмэна. Авторство тут ещё и расхуярено по полной, нужно было коммитить с: А так похуй. :roflanEbalo: |
Простите так нам удобнее апстримиться |
Ports of following PRs:
Goob-Station/Goob-Station#389
Goob-Station/Goob-Station#391
Стаки труб возвращаются!
Summary by CodeRabbit
Новые возможности
StrictPipeStackingдля управления правилами размещения труб.Улучшения
Конфигурация